Skip to content

Feat: Add CM Qube 500 Case#2612

Merged
limetech merged 3 commits intounraid:masterfrom
Squidly271:Feat-Add-CM-Qube-500-case
Apr 22, 2026
Merged

Feat: Add CM Qube 500 Case#2612
limetech merged 3 commits intounraid:masterfrom
Squidly271:Feat-Add-CM-Qube-500-case

Conversation

@Squidly271
Copy link
Copy Markdown
Contributor

@Squidly271 Squidly271 commented Apr 13, 2026 •
edited by coderabbitai Bot
Loading

Summary by CodeRabbit

  • Style
    • Added CM-Qube-500 case icon to the interface
    • Updated icon font assets

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 13, 2026 •
edited
Loading

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 777a4388-0090-4ba5-8ad7-fbc82919eb5b

📥 Commits

Reviewing files that changed from the base of the PR and between c83718a and 3fc4c72.

⛔ Files ignored due to path filters (1)
  • emhttp/plugins/dynamix/styles/font-cases.woff is excluded by !**/*.woff
📒 Files selected for processing (1)
  • emhttp/plugins/dynamix/styles/default-cases.css
🚧 Files skipped from review as they are similar to previous changes (1)
  • emhttp/plugins/dynamix/styles/default-cases.css

Walkthrough

A font-face source URL for the cases font was updated with a new version number, and a new CSS glyph mapping for the CM-Qube-500 case model icon was added.

Changes

Cohort / File(s) Summary
Font and Icon Glyph Updates
emhttp/plugins/dynamix/styles/default-cases.css		 Updated @font-face src from version 20260209 to 20260413, and added new .case-CM-Qube-500:before glyph mapping with content \e94b.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A new case model hops into view,
With CM-Qube-500's icon so true,
The font refreshed to version new,
E-nine-four-bee shines through and through! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Feat: Add CM Qube 500 Case' directly describes the main change in the pull request, which adds support for a new case model.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 13, 2026 •
edited
Loading

🔧 PR Test Plugin Available

A test plugin has been generated for this PR that includes the modified files.

Version: 2026.04.13.2121
Build: View Workflow Run

📥 Installation Instructions:

Install via Unraid Web UI:

  1. Go to Plugins → Install Plugin
  2. Copy and paste this URL:
https://preview.dl.unraid.net/pr-plugins/pr-2612/webgui-pr-2612.plg
  1. Click Install

Alternative: Direct Download

⚠️ Important Notes:

  • Testing only: This plugin is for testing PR changes
  • Backup included: Original files are automatically backed up
  • Easy removal: Files are restored when plugin is removed
  • Conflicts: Remove this plugin before installing production updates
  • Post-merge behavior: This preview stays available after merge until preview storage expires or it is manually cleaned up

📝 Modified Files:

Click to expand file list 
emhttp/plugins/dynamix/styles/default-cases.css
emhttp/plugins/dynamix/styles/font-cases.woff

🔄 To Remove:

Navigate to Plugins → Installed Plugins and remove webgui-pr-2612, or run:

plugin remove webgui-pr-2612

🤖 This comment is automatically generated and will be updated with each new push to this PR.

coderabbitai[bot]
coderabbitai Bot reviewed Apr 13, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents 
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@emhttp/plugins/dynamix/styles/default-cases.css`:
- Line 1: Update the pre-auth asset reference so the login page loads the new
font filename: change the reference to the cases font in emhttp/auth-request.php
from "/webGui/styles/font-cases.woff" to
"/webGui/styles/font-cases-20260413.woff" (or otherwise make both files
reference the same filename used in default-cases.css); search auth-request.php
for any other "/webGui/styles/font-cases" occurrences and update them to match
the new versioned name so pre-auth pages load the same font asset.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 56ed3314-e1ae-404b-a81d-de31f294da77

📥 Commits

Reviewing files that changed from the base of the PR and between 7756908 and 25b7942.

⛔ Files ignored due to path filters (1)
  • emhttp/plugins/dynamix/styles/font-cases-20260413.woff is excluded by !**/*.woff
📒 Files selected for processing (1)
  • emhttp/plugins/dynamix/styles/default-cases.css

Comment thread emhttp/plugins/dynamix/styles/default-cases.css Outdated
@Squidly271 Squidly271 added 7.2 7.3 and removed 7.2 labels Apr 13, 2026
@limetech limetech merged commit bec8f0b into unraid:master Apr 22, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@coderabbitai coderabbitai[bot] coderabbitai[bot] left review comments

Assignees

No one assigned

Labels

7.3

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Squidly271 @limetech